Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Skip issue matching check for thirdparty build dependency librar… #2990

Merged
merged 2 commits into from
Oct 8, 2024

Conversation

ebadiere
Copy link
Collaborator

Skip issue check for thirdparty libraries. We do not control them.

Related issue(s):

Fixes #2988

@ebadiere ebadiere requested review from a team as code owners September 16, 2024 17:21
@ebadiere ebadiere self-assigned this Sep 16, 2024
@ebadiere ebadiere added the dependencies Pull requests that update a dependency file label Sep 16, 2024
@ebadiere ebadiere added this to the 0.57.0 milestone Sep 16, 2024
Copy link

github-actions bot commented Sep 16, 2024

Tests

       3 files     297 suites   19s ⏱️
1 353 tests 1 352 ✔️ 1 💤 0
1 362 runs  1 361 ✔️ 1 💤 0

Results for commit 72291d1.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Sep 16, 2024

Acceptance Tests

  22 files  290 suites   34m 11s ⏱️
604 tests 588 ✔️ 4 💤 12
971 runs  954 ✔️ 4 💤 13

Results for commit 72291d1.

♻️ This comment has been updated with latest results.

Copy link
Member

@quiet-node quiet-node left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don’t think we should allow these third-party PRs to skip the check_pr CI task. Instead, we should let the CI fail for these PRs to serve as a reminder to update their labels and milestones.

… in the repository and not matching PRs with issues from third party projects.

Signed-off-by: ebadiere <[email protected]>
Copy link

sonarcloud bot commented Oct 4, 2024

Copy link
Member

@quiet-node quiet-node left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 1 nit can be addresed in future

.github/scripts/check-pr.js Show resolved Hide resolved
@ebadiere ebadiere merged commit 43e3ddf into main Oct 8, 2024
43 checks passed
@ebadiere ebadiere deleted the 2988-PR-check-ignore-issues-check-external-projects branch October 8, 2024 15:33
Copy link

codecov bot commented Oct 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.83%. Comparing base (f8e9e89) to head (72291d1).
Report is 42 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2990      +/-   ##
==========================================
- Coverage   89.54%   84.83%   -4.71%     
==========================================
  Files          56       59       +3     
  Lines        3796     3937     +141     
  Branches      775      786      +11     
==========================================
- Hits         3399     3340      -59     
- Misses        350      357       +7     
- Partials       47      240     +193     
Flag Coverage Δ
relay 84.94% <ø> (-4.75%) ⬇️
server 83.43% <ø> (-4.73%) ⬇️
ws-server 97.87% <ø> (-2.13%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 41 files with indirect coverage changes

ebadiere added a commit that referenced this pull request Oct 8, 2024
#2990)

* fix: Skip issue matching check for thirdparty build dependency libraries.
We do not control them.

Signed-off-by: ebadiere <[email protected]>

* fix: Now validates PRs listed in description by matching contributors in the repository and not matching PRs with issues from third party projects.

Signed-off-by: ebadiere <[email protected]>

---------

Signed-off-by: ebadiere <[email protected]>
ebadiere added a commit that referenced this pull request Oct 9, 2024
…82.0 (#3076)

* fix: upgrade @graphprotocol/graph-cli from 0.81.0 to 0.82.0

Snyk has created this PR to upgrade @graphprotocol/graph-cli from 0.81.0 to 0.82.0.

See this package in npm:
@graphprotocol/graph-cli

See this project in Snyk:
https://app.snyk.io/org/json-rpc-relay/project/24dd80bc-cab7-4d83-83d7-a604d57322e3?utm_source=github&utm_medium=referral&page=upgrade-pr

Signed-off-by: ebadiere <[email protected]>

* fix: Skip issue matching check for thirdparty build dependency librar… (#2990)

* fix: Skip issue matching check for thirdparty build dependency libraries.
We do not control them.

Signed-off-by: ebadiere <[email protected]>

* fix: Now validates PRs listed in description by matching contributors in the repository and not matching PRs with issues from third party projects.

Signed-off-by: ebadiere <[email protected]>

---------

Signed-off-by: ebadiere <[email protected]>

* fix: SDKClient timeouts are not getting logged with requestIds. (#3061)

* fix: Enhanced logged warning to include SDKClient error message and bumped up SDK_REQUEST_TIMEOUT for CI.

Signed-off-by: ebadiere <[email protected]>

* fix: Removed the increate in sdkClient timeout.

Signed-off-by: ebadiere <[email protected]>

---------

Signed-off-by: ebadiere <[email protected]>

---------

Signed-off-by: ebadiere <[email protected]>
Co-authored-by: snyk-bot <[email protected]>
Co-authored-by: Eric Badiere <[email protected]>
konstantinabl pushed a commit that referenced this pull request Oct 15, 2024
#2990)

* fix: Skip issue matching check for thirdparty build dependency libraries.
We do not control them.

Signed-off-by: ebadiere <[email protected]>

* fix: Now validates PRs listed in description by matching contributors in the repository and not matching PRs with issues from third party projects.

Signed-off-by: ebadiere <[email protected]>

---------

Signed-off-by: ebadiere <[email protected]>
konstantinabl pushed a commit that referenced this pull request Oct 15, 2024
…82.0 (#3076)

* fix: upgrade @graphprotocol/graph-cli from 0.81.0 to 0.82.0

Snyk has created this PR to upgrade @graphprotocol/graph-cli from 0.81.0 to 0.82.0.

See this package in npm:
@graphprotocol/graph-cli

See this project in Snyk:
https://app.snyk.io/org/json-rpc-relay/project/24dd80bc-cab7-4d83-83d7-a604d57322e3?utm_source=github&utm_medium=referral&page=upgrade-pr

Signed-off-by: ebadiere <[email protected]>

* fix: Skip issue matching check for thirdparty build dependency librar… (#2990)

* fix: Skip issue matching check for thirdparty build dependency libraries.
We do not control them.

Signed-off-by: ebadiere <[email protected]>

* fix: Now validates PRs listed in description by matching contributors in the repository and not matching PRs with issues from third party projects.

Signed-off-by: ebadiere <[email protected]>

---------

Signed-off-by: ebadiere <[email protected]>

* fix: SDKClient timeouts are not getting logged with requestIds. (#3061)

* fix: Enhanced logged warning to include SDKClient error message and bumped up SDK_REQUEST_TIMEOUT for CI.

Signed-off-by: ebadiere <[email protected]>

* fix: Removed the increate in sdkClient timeout.

Signed-off-by: ebadiere <[email protected]>

---------

Signed-off-by: ebadiere <[email protected]>

---------

Signed-off-by: ebadiere <[email protected]>
Co-authored-by: snyk-bot <[email protected]>
Co-authored-by: Eric Badiere <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

The PR check in CI needs to be updated to ignore the issues check on external dependencies.
3 participants